[#516] Set story title character limit to 60 characters#522
[#516] Set story title character limit to 60 characters#522realproject7 merged 1 commit intomainfrom
Conversation
- Enforce max 60 chars on new storyline title input (maxLength + slice) - Show character counter (e.g. "42 / 60") below title field - Prevent submission when title exceeds limit - Bump version to 0.1.1 Fixes #516 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
APPROVE — clean, focused implementation. Double enforcement (maxLength + slice) is solid. Character counter and validation logic are correct. Minor nit: MAX_TITLE_LENGTH could be a module-level constant instead of inside the component body, but not blocking.
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
This PR meets issue #516: the create storyline title input now enforces a 60-character limit, shows a live counter, and keeps submission blocked when the title is invalid or empty. The change is minimal, follows the existing title-input pattern on the page, and includes the required patch version bump.
Findings
- None.
Decision
Approving because the diff satisfies the acceptance criteria without introducing scope creep. CI checks are still pending, but that is not blocking this code review.
2 participants
Summary
maxLengthand.slice(0, 60){length} / 600.1.1Test plan
X / 60Fixes #516
🤖 Generated with Claude Code